Skip to content

feat(server): MBTI 결과 저장/통계 API 추가#305

Merged
CSE-Shaco merged 1 commit into
GDGoCINHA:developfrom
CSE-Shaco:develop
Mar 9, 2026
Merged

feat(server): MBTI 결과 저장/통계 API 추가#305
CSE-Shaco merged 1 commit into
GDGoCINHA:developfrom
CSE-Shaco:develop

Conversation

@CSE-Shaco

@CSE-Shaco CSE-Shaco commented Mar 9, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

📌 연관된 이슈

ex) #이슈번호, #이슈번호

✨ 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요

💬 리뷰 요구사항(선택)

Summary by CodeRabbit

릴리스 노트

새로운 기능

  • MBTI 성격 유형 검사 결과를 저장하고 업데이트하는 기능이 추가되었습니다.
  • MBTI 유형별 참여자 분포 및 통계 정보를 조회할 수 있게 되었습니다.

@coderabbitai

coderabbitai Bot commented Mar 9, 2026 •
edited
Loading

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

MBTI 결과 관리 기능을 추가합니다. 신규 엔티티, 저장소, 서비스, 컨트롤러 엔드포인트, 데이터 전송 객체 및 데이터베이스 마이그레이션을 포함합니다.

Changes

Cohort / File(s) Summary
Controller & Messages
GameUserController.java, GameUserMessage.java		 MbtiResultService 의존성 추가, POST /mbti/result (upsert), GET /mbti/result/stats 엔드포인트 추가, 성공 메시지 상수 2개 추가
Request/Response DTOs
MbtiResultRequest.java, MbtiResultResponse.java, MbtiStatsResponse.java, MbtiTypeCountResponse.java		 MBTI 결과 요청/응답 및 통계 응답 DTO 4개 신규 추가, 엔티티 매핑 및 생성자 포함
Entity & Repository
MbtiResult.java, MbtiResultRepository.java, MbtiTypeCountProjection.java		 JPA 엔티티 MbtiResult 신규 (name, studentId, mbtiType 필드, 유니크 제약), 복합 조회 및 집계 쿼리 메서드 포함 저장소, 타입 카운트 프로젝션 인터페이스
Service
MbtiResultService.java		 upsertMbtiResult() (업데이트 또는 신규 저장), getMbtiStats() (총 개수 및 유형별 카운트 조회) 메서드 2개 포함 서비스 클래스
Database Migration
V20260306__create_mbti_result.sql		 mbti_result 테이블 생성 (id, name, student_id, mbti_type, created_at, updated_at), name과 student_id의 유니크 인덱스 추가

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested labels

enhancement

Poem

🐰 숲속 토끼가 전하는 축하의 말씀:
MBTI 결과를 모아 모아,
테이블에 쏘옥 저장하고,
통계를 산으로 만들어,
당신의 성격 유형을 선명하게 밝혔네! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경 사항의 주요 내용을 명확하게 설명합니다. MBTI 결과 저장 및 통계 API 추가는 제공된 코드 변경 사항과 완벽하게 일치합니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@CSE-Shaco CSE-Shaco merged commit 9d3d36b into GDGoCINHA:develop Mar 9, 2026
1 of 2 checks passed
This was referenced Mar 9, 2026
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@CSE-Shaco